cmd/collectors/eseries/rest/client.go:269

• For non-200 responses (other than 401), the returned RestError is built with only StatusCode and API, and the prior log that included HTTP status/body was removed. This will make failures hard to diagnose. Consider including at least Message(res.Status) (and optionally a sanitized/truncated response body) in the error, or re-adding structured logging at an appropriate level.

		if res.StatusCode != http.StatusOK {
			return nil, errs.NewRest().
				StatusCode(res.StatusCode).
				API(endpoint).
				Build()
		}

cmd/collectors/eseriesperf/eseriesperf.go:205

• This if _, exists := ep.Prop.Metrics[timestampMetric]; !exists { ... } block is unreachable because timestampMetric is selected by iterating over ep.Prop.Metrics keys right above, so it will always exist when non-empty. Consider simplifying by removing the redundant existence check (or set timestampMetric from a different source if that was the intent).

	// Fallback: look for timestamp metric (e.g., statistics.timestamp for SSD cache)
	if timestampMetric == "" {
		for metricName := range ep.Prop.Metrics {
			if strings.HasSuffix(metricName, "timestamp") {
				timestampMetric = metricName
				break
			}
		}
	}

	if timestampMetric != "" {
		if _, exists := ep.Prop.Metrics[timestampMetric]; !exists {
			ep.Prop.Metrics[timestampMetric] = &eseries.Metric{
				Label:      "observedTimeInMS",
				Name:       timestampMetric,
				Exportable: false,
			}
		}
	}

cmd/collectors/eseries/rest/client.go:269

• For non-200 (non-401) responses, the prior structured log with status, url, and response body has been removed. Since the returned RestError currently includes only StatusCode and API, this makes troubleshooting API failures significantly harder in production. Consider restoring some form of diagnostic context (e.g., include res.Status/body in RestError.Message, or log it at debug level with redaction if needed).

		}

		if res.StatusCode != http.StatusOK {
			return nil, errs.NewRest().
				StatusCode(res.StatusCode).
				API(endpoint).
				Build()
		}